Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Contao 4.13 support. Wrong Tax. Klarna Password Length. #14

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Contao 4.13 support. Wrong Tax. Klarna Password Length. #14

wants to merge 5 commits into from

Conversation

d0p3fish
Copy link

@d0p3fish
Update Surcharge.php
b6bdabb 
fix tax_rate in Surcharge
@d0p3fish
Update ShippingOption.php
4f580a6 
fix tax_amount in ShippingOption Dto
@d0p3fish
Update composer.json
7dbd085 
doctrine/dbal v3 in composer.json
@d0p3fish
Update tl_iso_config.php
1ab8294 
extend password length, because of new passwords around 180 chars long
@d0p3fish
Update composer.json
98b5d37 
dbal v2 or v3
@richardhj
Copy link
Owner

Hi @d0p3fish, dank dir!

Kannst du die Änderungen nochmal erläutern?

  • Sind die Werte durch die Änderungen des Multiplikator (Einmal fällt *100 weg / Einmal wird *1000 ergänzt) immer noch die selben?
  • Welcher Wert wurde vorher ($this->tax_rate) genommen, warum muss es $surcharge-> sein?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@d0p3fish @richardhj